Skip to content

U/sgriffin/vc2026#167

Open
stephenegriffin wants to merge 26 commits intomainfrom
u/sgriffin/vc2026
Open

U/sgriffin/vc2026#167
stephenegriffin wants to merge 26 commits intomainfrom
u/sgriffin/vc2026

Conversation

@stephenegriffin
Copy link
Member

No description provided.

- Create Directory.Build.props for centralized build configuration
- Update solution file to VS 18 format
- Update .vsconfig to require Windows 11 SDK (10.0.22621.0)
- Remove PlatformToolset and WindowsTargetPlatformVersion from mapistub.vcxproj
- Remove Guardian .gdn ruleset references (CI-only dependency)

Breaking change: Requires Visual Studio 2026
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 776c362. ± Comparison against base commit 7650ee0.

♻️ This comment has been updated with latest results.

stephenegriffin and others added 6 commits February 6, 2026 18:14
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Windows/Visual Studio build setup to target a newer toolset/SDK baseline and align CI/security scanning workflows with those build requirements.

Changes:

  • Centralizes MSBuild defaults (toolset/SDK pin + security/quality flags) via Directory.Build.props and removes per-configuration duplication from the .vcxproj.
  • Updates the solution/project configurations (drops Prefast configs) and bumps the .sln Visual Studio version metadata.
  • Updates GitHub Actions workflows (new Windows runner label, ARM64EC matrix, Windows 11 SDK install step) and bumps CodeQL action SHAs.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
mapistub.vcxproj Removes per-config toolset/SDK/security flags and drops Prefast/Fuzz ARM64* configurations in favor of repo-wide props.
mapistub.sln Updates VS version metadata and removes Prefast solution configurations.
Directory.Build.props Introduces centralized toolset/SDK pin and baseline compiler/link security settings.
.vsconfig Updates required SDK component to Windows 11 SDK 22621.
.github/workflows/github-ci.yml Switches to a VS2026 runner label, adds ARM64EC to the build matrix, and installs Windows 11 SDK.
.github/workflows/codeql.yml Switches to a VS2026 runner label, installs Windows 11 SDK, and updates CodeQL action SHAs.
.github/workflows/scorecards.yml Updates upload-sarif action SHA.
.github/workflows/devskim.yml Updates upload-sarif action SHA.
.github/workflows/clang.yml Replaces the clang-format action with an apt-based install + manual formatting check script.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

stephenegriffin and others added 5 commits February 17, 2026 17:48
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 9 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (1)

mapistub.sln:24

  • The solution file is missing Fuzz configurations for Win32 and x64, but these configurations still exist in the mapistub.vcxproj file (lines 23-28, 103-109, 135-141). This inconsistency will prevent building Fuzz configurations from the solution. Either add Fuzz|Win32 and Fuzz|x64 to the solution file, or remove these configurations from the project file as well.
	GlobalSection(SolutionConfigurationPlatforms) = preSolution
		Debug_Unicode|x64 = Debug_Unicode|x64
		Debug_Unicode|Win32 = Debug_Unicode|Win32
		Debug_Unicode|ARM64 = Debug_Unicode|ARM64
		Debug_Unicode|ARM64EC = Debug_Unicode|ARM64EC
		Debug|x64 = Debug|x64
		Debug|Win32 = Debug|Win32
		Debug|ARM64 = Debug|ARM64
		Debug|ARM64EC = Debug|ARM64EC
		Release_Unicode|x64 = Release_Unicode|x64
		Release_Unicode|Win32 = Release_Unicode|Win32
		Release_Unicode|ARM64 = Release_Unicode|ARM64
		Release_Unicode|ARM64EC = Release_Unicode|ARM64EC
		Release|x64 = Release|x64
		Release|Win32 = Release|Win32
		Release|ARM64 = Release|ARM64
		Release|ARM64EC = Release|ARM64EC

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

stephenegriffin and others added 3 commits February 24, 2026 16:25
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants